-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Privileged LDAP (again) #169
base: master
Are you sure you want to change the base?
Conversation
I think you want to take another look at this. |
well this is not ideal |
This reverts commit bec7635.
@ja5087 I added a commit. I think now you will just need to fix the tests to account for the different functions. Sorry for all the trouble. Reverting merges in git is known to be annoying and hard to handle. |
What's the current status of this? |
It's been stalled for a while, but I do have a better understanding of the system now and could get back on it soon |
@ja5087 would you have time to re-base and bring this over the finish line at some point? Would love to see this landed! |
I can potentially finish this at some point if I still have time, but I'm unsure if password is used frequently as what I encountered mostly is that we use Kerberos ticket to auth??? |
Last time we tried to merge this (see #157) the prerequisite ucbldap.passwd was not present in ocfweb, dev-ocfweb etc.
I've rolled back all the changes I made to the tests. Once the password file has been added to the puppet configs for ocfweb and Jenkins, this PR should be good for merging.